Skip to content

Publish to PyPI on version tag push#5

Merged
eddietejeda merged 1 commit into
mainfrom
ci/publish-on-version-tags
May 19, 2026
Merged

Publish to PyPI on version tag push#5
eddietejeda merged 1 commit into
mainfrom
ci/publish-on-version-tags

Conversation

@eddietejeda
Copy link
Copy Markdown
Contributor

Summary

  • Add a Publish to PyPI workflow triggered by version tags matching v[0-9]*
  • Verify the tag matches pyproject.toml, build sdist/wheel, and upload via PyPI Trusted Publishing

Test plan

  • Configure PyPI Trusted Publishing for hotdata-runtime (GitHub environment pypi)
  • Merge this PR
  • Bump version in pyproject.toml, push tag (e.g. git tag v0.1.0 && git push origin v0.1.0)
  • Confirm workflow succeeds and package appears at https://pypi.org/p/hotdata-runtime

Add a tag-triggered publish workflow matching sdk-python: build, verify
the tag matches pyproject.toml, and upload via PyPI Trusted Publishing.
Comment on lines +43 to +52
- name: Build sdist and wheel
run: python -m build

- name: Check distribution metadata
run: python -m twine check --strict dist/*

- uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5
with:
name: dist
path: dist/
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

super nit: there is no CI workflow in this repo, so a tag-push goes straight from "build" to "publish to PyPI" without ever running pytest. Consider adding a test step (e.g. uv sync && uv run pytest) before python -m build, or gate publishing on a separate CI workflow having passed on the tagged commit. Otherwise a broken commit that happens to build cleanly will land on PyPI. (not blocking)

@eddietejeda eddietejeda merged commit ba9934c into main May 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant